Set promise out of ExchangeQueue lock#4111
Set promise out of ExchangeQueue lock#4111xiaoxmeng wants to merge 1 commit intofacebookincubator:mainfrom
Conversation
✅ Deploy Preview for meta-velox canceled.
|
|
@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
This pull request was exported from Phabricator. Differential Revision: D43504994 |
|
This pull request was exported from Phabricator. Differential Revision: D43504994 |
|
This pull request was exported from Phabricator. Differential Revision: D43504994 |
|
This pull request was exported from Phabricator. Differential Revision: D43504994 |
Summary: X-link: prestodb/presto#19101 Set promise out of ExchangeQueue lock to avoid potential deadlock caused by the inline executed future continuation callback Pull Request resolved: facebookincubator#4111 Reviewed By: Yuhta Differential Revision: D43504994 Pulled By: xiaoxmeng fbshipit-source-id: 46c9839307e8c97a238cb1ecc7d74c0c15dcfc5a
|
This pull request was exported from Phabricator. Differential Revision: D43504994 |
|
@xiaoxmeng merged this pull request in 92fbfb9. |
Set promise out of ExchangeQueue lock to avoid potential deadlock
caused by the inline executed future continuation callback